Skip to content
This repository was archived by the owner on Jan 11, 2020. It is now read-only.

Prefetch participant images fixes #311#351

Open
anupm12 wants to merge 6 commits into
opencodeiiita:masterfrom
anupm12:prefetch-images
Open

Prefetch participant images fixes #311#351
anupm12 wants to merge 6 commits into
opencodeiiita:masterfrom
anupm12:prefetch-images

Conversation

@anupm12

@anupm12 anupm12 commented Feb 3, 2019

Copy link
Copy Markdown
Collaborator
  • Please fill out the template below

Did you read the instructions in README ?

Yes

Solves which issue ?

Fixes #311

Give discription.

Images are now being cached by the browser and the loading is comparatively faster now. All time it takes is to fetch data from json.
I have made a script to fetch the images and prefetched that script using HTML.

Checklist

(To mark the checkboxes use x within the box brackets)

  • Signed the code of conduct.

  • Have you added yourself to the participants.

  • Joined opencode zulip server.

@opencodebot opencodebot Bot requested a review from Anupam-dagar February 3, 2019 06:28
@anupm12

anupm12 commented Feb 3, 2019

Copy link
Copy Markdown
Collaborator Author

@Anupam-dagar @ananya-agrawal please review this pr.

@anupm12

anupm12 commented Feb 3, 2019

Copy link
Copy Markdown
Collaborator Author

I'm not sure, but I think it works better in Firefox developer edition.

@prateek76

Copy link
Copy Markdown
Contributor

link

@anupm12

anupm12 commented Feb 3, 2019

Copy link
Copy Markdown
Collaborator Author

I'm sorry. I forget everytime 😅
live link
https://preloadimages.netlify.com/

@prateek76

Copy link
Copy Markdown
Contributor

@anupm12 i can't see any impovement also in firefox it's taking more time than before

@anupm12

anupm12 commented Feb 3, 2019

Copy link
Copy Markdown
Collaborator Author

@prateek76
Let me check it

@ananya ananya requested a review from pradeepgangwar February 4, 2019 13:34
@anupm12

anupm12 commented Feb 4, 2019

Copy link
Copy Markdown
Collaborator Author

@pradeepgangwar @ananya-agrawal @prateek76
I have made use of prefetch HTML attribute and I read on StackOverflow that this attribute is not widely supported by browsers.
Can you suggest me some alternative method to prefetch images folder or the participant.json file?

https://stackoverflow.com/questions/41655955/why-are-preload-link-not-working-for-json-requests

@anupm12

anupm12 commented Feb 4, 2019

Copy link
Copy Markdown
Collaborator Author

This is my code
image

This is what it does
image

I don't know if this is preloading or not.

@Anupam-dagar

Copy link
Copy Markdown
Member

@pradeepgangwar take a look at it.

@pradeepgangwar

Copy link
Copy Markdown
Member

This is not the correct way to load image and assign. I see that in your script you are appending it in body tag.
I will get back to this today evening.

@anupm12

anupm12 commented Feb 5, 2019

Copy link
Copy Markdown
Collaborator Author

Ok @pradeepgangwar

@pradeepgangwar

Copy link
Copy Markdown
Member

@anupm12 Sorry for delay 😅 . I got busy with other things and completely forgot about this one. Let me check this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants